Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Test videos with B-Frames #1157

Merged
merged 1 commit into from
Jul 24, 2019
Merged

Test videos with B-Frames #1157

merged 1 commit into from
Jul 24, 2019

Conversation

fmassa
Copy link
Member

@fmassa fmassa commented Jul 24, 2019

Also extend video saving to support different codecs and options. Notably, we can now save with lossless compression

Also extend video saving to support different codecs and options. Notably, we can now save with lossless compression
@codecov-io
Copy link

codecov-io commented Jul 24, 2019

Codecov Report

Merging #1157 into master will decrease coverage by 0.04%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #1157      +/-   ##
==========================================
- Coverage   65.65%   65.61%   -0.05%     
==========================================
  Files          70       70              
  Lines        5527     5529       +2     
  Branches      853      853              
==========================================
- Hits         3629     3628       -1     
- Misses       1639     1641       +2     
- Partials      259      260       +1
Impacted Files Coverage Δ
torchvision/io/video.py 73.95% <100%> (+2.68%) ⬆️
torchvision/transforms/transforms.py 80.55% <0%> (-0.99%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update fe3b4c8...8bf75f0. Read the comment docs.

@fmassa fmassa merged commit 010984d into pytorch:master Jul 24, 2019
@fmassa fmassa deleted the test-bframes branch July 24, 2019 14:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants